revert: remove client-side Sentry (#642)#647
Conversation
Reverts the gated client-side @sentry/browser integration merged in #642: tunnel route (/api/monitoring), client init/scrubber/holder, error-boundary capture wiring, NEXT_PUBLIC_SENTRY_DSN env, the DefinePlugin gate, and the @sentry/browser dependency. Server-side error capture (@sentry/node via src/lib/observability/error-capture.ts + src/instrumentation.ts, gated on SENTRY_DSN) is unaffected — it predates #642. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (11)
📝 WalkthroughWalkthroughThe PR removes client-side Sentry reporting, its browser dependency and configuration, the monitoring API route, and the related environment variables. Error boundaries retain console logging and existing fallback UI behavior. ChangesSentry integration removal
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
Superseded by #645 ( |
Summary
Reverts the client-side Sentry integration merged in #642, as requested. Removes:
@sentry/browserdependencysrc/app/api/monitoring/route.tssrc/lib/observability/sentry-client.ts+sentry-scrub.tsandtests/sentry-client.test.tssrc/instrumentation-client.ts(back to just the Zod jitless config) and the__SENTRY_ENABLED__DefinePlugin innext.config.tsNEXT_PUBLIC_SENTRY_DSN/NEXT_PUBLIC_SENTRY_ENVIRONMENTfromenv.tscaptureClientExceptionwiring in the route + global error boundariesServer-side error capture is untouched —
@sentry/nodeviasrc/lib/observability/error-capture.ts+src/instrumentation.ts(gated onSENTRY_DSN) predates #642 and remains, fully inert without a DSN.Done as a clean
git revertof the #642 squash commit (71d9f2c76); no conflicts.Verification
typecheck,lint— cleanbuild— exit 0; no@sentry/sentry-scrub/sentry-client/__SENTRY_ENABLED__in the client bundle;/api/monitoringroute absent from build outputtest— 2298 pass (the 3 "failures" wereTest timed outflakes from a concurrent build starving the AST-graph tests; both files pass 10/10 when re-run in isolation)src/tests/next.config.tsClinical Governance Preflight
N/A — removes an observability integration; no ingestion, answer generation, search, privacy, or clinical output behavior changes. (The client tunnel and its public DSN are gone; server capture behavior is unchanged.)
🤖 Generated with Claude Code
Summary by CodeRabbit
Changes
Documentation